Skip to content

Update for "rsz: Convert buffering to fixed-point delays"#3281

Merged
maliberty merged 3 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:bump-or-250207
Jul 2, 2025
Merged

Update for "rsz: Convert buffering to fixed-point delays"#3281
maliberty merged 3 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:bump-or-250207

Conversation

@openroad-ci
Copy link
Copy Markdown
Collaborator

@openroad-ci openroad-ci commented Jul 2, 2025

This pulls in OR changes including The-OpenROAD-Project/OpenROAD#7648 and addresses two issues surfacing due to the resizer change

Updated Rules

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
designs/ihp-sg13g2/aes/rules-base.json updates:

Metric Old New Type
detailedroute__antenna_diodes_count 5 12 Failing

designs/ihp-sg13g2/jpeg/rules-base.json updates:

Metric Old New Type
globalroute__antenna_diodes_count 4 12 Failing

designs/nangate45/bp_fe_top/rules-base.json updates:

Metric Old New Type
finish__timing__setup__ws -0.12 -0.21 Failing

designs/sky130hd/gcd/rules-base.json updates:

Metric Old New Type
detailedroute__route__wirelength 9752 11675 Failing
finish__timing__drv__setup_violation_count 81 86 Failing

[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000.
designs/sky130hd/riscv32i/rules-base.json updates:

Metric Old New Type
globalroute__antenna_diodes_count 10 20 Failing

Messages from CI

[INFO] asap7/cva6 not included in CI.
[INFO] asap7/minimal not included in CI.
[INFO] asap7/mock-cpu set to not run for this PR.
[INFO] asap7/swerv_wrapper set to not run for this PR.
[INFO] gf12 not included in the update.
[INFO] gf55 not included in the update.
[INFO] nangate45/ariane136 set to not run for this PR.
[INFO] nangate45/black_parrot set to not run for this PR.
[INFO] nangate45/bp_quad not included in CI.
[INFO] nangate45/mempool_group not included in CI.
[INFO] src not included in the update.

povik added 2 commits July 2, 2025 14:13
Signed-off-by: Martin Povišer <povik@cutebit.org>
Signed-off-by: Martin Povišer <povik@cutebit.org>

export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NAME)/fastroute.tcl

export GPL_KEEP_OVERFLOW = 0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is to work around placer divergence. I've opened #3282

@povik povik added the UpdateRules Starts GHA to update rules label Jul 2, 2025
@maliberty maliberty enabled auto-merge July 2, 2025 15:39
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label Jul 2, 2025
@maliberty maliberty disabled auto-merge July 2, 2025 19:11
@maliberty maliberty merged commit 3d39210 into The-OpenROAD-Project:master Jul 2, 2025
6 of 8 checks passed
@maliberty maliberty deleted the bump-or-250207 branch July 2, 2025 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants